Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change search to match every sub part #1054

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MSchmoecker
Copy link

@MSchmoecker MSchmoecker commented Aug 24, 2024

The search in Thunderstore is currently barely usable for searches with more than one term.

Instead of matching any part of the query in any field, this matches every part in any field. I.e. all parts must be in the name, description or author name, not only one part. Similar how r2modman works.

For example in Valheim, searching 'Epic Loot' returns about 100 packages (4 pages), with the actual package burred in the last page because it is named 'EpicLoot'. Most of the other packages are completely unrelated but either contain 'Epic' or 'Loot' somewhere.
With this PR, the result is only about 6 packages related and including EpicLoot (in my dev environment that is populated with Valheim package names).

Overall, this makes the search more strict. However, this is usually the expected behaviour. Adding more terms to the search should narrow down the results, not give more matches.

@MSchmoecker
Copy link
Author

@MythicManiac any thoughts about this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant